Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: add imageio-ffmpeg requirement to install.py #42

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

neggles
Copy link
Contributor

@neggles neggles commented Mar 15, 2023

Fixes #41 by adding imageio-ffmpeg to extension requirements/dependencies.

This is required due to some changes with moviepy's underlying imageio library, which has deprecated/removed the old autoinstallation method around 2019-ish, and as such moviepy is no longer capable of autoinstalling ffmpeg.

See Zulko/moviepy#906 and Zulko/moviepy#908 for further details on the moviepy side.

This does result in downloading ffmpeg on systems where it may already be installed, but it's a reasonably small download and on some systems (eg Ubuntu 22.04) moviepy 1.0.3 fails to pick up system ffmpeg anyway.

Fixes yownas#41 by adding imageio-ffmpeg to extension requirements/dependencies.

This is required due to some changes with moviepy's underlying imageio
library, which has deprecated/removed the old autoinstallation method
around 2019-ish, and as such moviepy is no longer capable of
autoinstalling ffmpeg.

See Zulko/moviepy#906 and Zulko/moviepy#908
for further details on the moviepy side.

This does result in downloading ffmpeg on systems where it may already
be installed, but it's a reasonably small download and on some systems
(eg Ubuntu 22.04) moviepy 1.0.3 fails to pick up system ffmpeg anyway.
@yownas yownas merged commit ebc887b into yownas:main Mar 15, 2023
@yownas
Copy link
Owner

yownas commented Mar 15, 2023

Thank you. :) I hope this will fix some of the problems people have had.

@neggles neggles deleted the imageio-ffmpeg branch March 15, 2023 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installation Issues
2 participants